Skip to content

Fix the CI & update the build#238

Merged
Gedochao merged 5 commits into
VirtusLab:mainfrom
Gedochao:maintainance/fix-build
Jun 23, 2026
Merged

Fix the CI & update the build#238
Gedochao merged 5 commits into
VirtusLab:mainfrom
Gedochao:maintainance/fix-build

Conversation

@Gedochao

Copy link
Copy Markdown

Mill 1.0.6 => 1.1.7
mill-native-image & mill-native-image-upload 0.2.4 => 0.2.6
coursier 2.1.25-M23 => 2.1.25-M25
This ought to fix the CI on Windows

Gedochao and others added 3 commits June 23, 2026 11:50
Rename .mill.scala build files to .mill as required by Mill 1.1.x, and
update Sonatype publish types to use mill.javalib.publish APIs.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@Gedochao
Gedochao requested review from tgodzik and zielinsky June 23, 2026 10:09

@tgodzik tgodzik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the new plugin is not working well either:


Error: Classes that should be initialized at run time got initialized during image building:
 org.bouncycastle.crypto.macs.HMac was unintentionally initialized at build time. To see why org.bouncycastle.crypto.macs.HMac got initialized use --trace-class-initialization=org.bouncycastle.crypto.macs.HMac
org.bouncycastle.crypto.CryptoServicesRegistrar was unintentionally initialized at build time. To see why org.bouncycastle.crypto.CryptoServicesRegistrar got initialized use --trace-class-initialization=org.bouncycastle.crypto.CryptoServicesRegistrar
------------------------------------------------------------------------------------------------------------------------
org.bouncycastle.util.Strings was unintentionally initialized at build time. To see why org.bouncycastle.util.Strings got initialized use --trace-class-initialization=org.bouncycastle.util.Strings
org.bouncycastle.util.Properties was unintentionally initialized at build time. To see why org.bouncycastle.util.Properties got initialized use --trace-class-initialization=org.bouncycastle.util.Properties
To see how the classes got initialized, use --trace-class-initialization=org.bouncycastle.crypto.macs.HMac,org.bouncycastle.crypto.CryptoServicesRegistrar,org.bouncycastle.util.Strings,org.bouncycastle.util.Properties
                        1.1s (3.2% of total time) in 21 GCs | Peak RSS: 2.97GB | CPU load: 3.44
========================================================================================================================
Failed generating 'output' after 32.6s.
Error: Use -H:+ReportExceptionStackTraces to print stacktrace of underlying exception
Error: Image build request failed with exit status 1

@Gedochao
Gedochao marked this pull request as draft June 23, 2026 10:43
@Gedochao
Gedochao force-pushed the maintainance/fix-build branch from 6182229 to 885cb92 Compare June 23, 2026 10:54
@Gedochao
Gedochao force-pushed the maintainance/fix-build branch from 885cb92 to 5994a05 Compare June 23, 2026 10:58
@Gedochao

Copy link
Copy Markdown
Author

Not so much the matter of the plugin, as a bouncycastle incompatibility.
Should be fixed now.

@Gedochao
Gedochao requested a review from tgodzik June 23, 2026 11:04
@Gedochao
Gedochao marked this pull request as ready for review June 23, 2026 11:04

@tgodzik tgodzik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Gedochao
Gedochao merged commit 9822f8a into VirtusLab:main Jun 23, 2026
11 checks passed
@Gedochao
Gedochao deleted the maintainance/fix-build branch June 23, 2026 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants